Skip to content

[ refactor ] Exchange components of Relation.Binary.Definitions._Respects₂_#2515

Open
jamesmckinna wants to merge 10 commits intoagda:masterfrom
jamesmckinna:issue2471
Open

[ refactor ] Exchange components of Relation.Binary.Definitions._Respects₂_#2515
jamesmckinna wants to merge 10 commits intoagda:masterfrom
jamesmckinna:issue2471

Conversation

@jamesmckinna
Copy link
Copy Markdown
Collaborator

@jamesmckinna jamesmckinna commented Dec 9, 2024

Fixes #2471

Lots of (trivial) knock-on viscosity: PR takes the opportunity to rename/refactor proofs in (hopefully!) helpful ways.

Is this v2.x or v3.0, because breaking?

@MatthewDaggitt MatthewDaggitt added this to the v3.0 milestone Dec 10, 2024
@MatthewDaggitt
Copy link
Copy Markdown
Collaborator

This is definitely v3.0 I'm afraid. Large breaking change...

Copy link
Copy Markdown
Collaborator

@JacquesCarette JacquesCarette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

v3.0 it might be, but here's my approval anyways.

@jamesmckinna
Copy link
Copy Markdown
Collaborator Author

Will resolve the merge conflict only once CHANGELOG moves from v2.2 to v2.3... but even that is likely to be premature ;-)

Comment thread CHANGELOG.md Outdated
@jamesmckinna
Copy link
Copy Markdown
Collaborator Author

I'll only return to the merge conflicts after v2.3 gets released (or else we fast-forward directly to v3.0)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Relation.Binary.Definitions._Respects₂_ seems to exchange 'left' and 'right' in its left/right projections?

4 participants